Skip to content

docs: simplify the review-fix-loop design - #94

Merged
shaug merged 1 commit into
mainfrom
scott/simplify-review-fix-loop-design
Jul 30, 2026
Merged

docs: simplify the review-fix-loop design#94
shaug merged 1 commit into
mainfrom
scott/simplify-review-fix-loop-design

Conversation

@shaug

@shaug shaug commented Jul 30, 2026

Copy link
Copy Markdown
Owner

TL;DR

Simplifies the review-fix-loop design into a standalone, locally coordinated skill and moves caller migrations into dependent fast-follow work.

Summary

  • Removes the proposed shared ownership provider, coordinator service, distributed leases, and fencing protocol.
  • Uses Git-common-directory locks, isolated remediation attempts, durable local checkpoints, and expected-old fast-forward publication.
  • Preserves fresh read-only aggregate reviewers, invocation-owned cycle budgets, local-until-converged fixes, stable terminal states, and caller-owned acceptance reconciliation.
  • Keeps implement-ticket, babysit-pr, carve-changesets, and other caller migrations outside the implementation epic.

Validation

  • just format
  • just lint
  • just test
  • Iterative fresh read-only design review, final result clean

## Summary

- Replace distributed ownership and coordinator machinery with local locking, isolated attempts, durable checkpoints, and Git-native publication guards
- Keep existing caller migrations outside the implementation epic as dependent fast follows
- Tighten reviewer isolation, source divergence, validation-failure, base-drift, and terminal-result contracts

## Why

- Keep the initial skill proportionate to its controlled operating environment
- Give the implementation epic a clear standalone boundary
@shaug
shaug merged commit 2e7a8cd into main Jul 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant